Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for specifying container hostname #889

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

kyzooghost
Copy link

Address #883

Added TestContainer.withHostname() and StartedTestContainer.getHostname()

* added withHostname

* added getHostname
Copy link

netlify bot commented Jan 8, 2025

Deploy Preview for testcontainers-node ready!

Name Link
🔨 Latest commit d1ded06
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-node/deploys/6792358c3bb11e00084b59c4
😎 Deploy Preview https://deploy-preview-889--testcontainers-node.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@cristianrgreco
Copy link
Collaborator

@kyzooghost Pls add an example to the docs, in containers.md

@cristianrgreco cristianrgreco added enhancement New feature or request minor Backward compatible functionality labels Jan 13, 2025
@cristianrgreco cristianrgreco changed the title [Feat] Added custom container hostname Add support for specifying container hostname Jan 13, 2025
@kyzooghost
Copy link
Author

@kyzooghost Pls add an example to the docs, in containers.md

added with a cautionary note, please check :)

@cristianrgreco
Copy link
Collaborator

cristianrgreco commented Jan 21, 2025

@kyzooghost The build is failing.

Probably the AbstractStartedContainer no longer fully implements the StartedTestContainer interface

@kyzooghost
Copy link
Author

@kyzooghost The build is failing.

Probably the AbstractStartedContainer no longer fully implements the StartedTestContainer interface

Will look into it when I can :)

@kyzooghost
Copy link
Author

@cristianrgreco pushed another commit with AbstractStartedContainer.getHostname() method

Ran npm run build -ws on my local and it looks good to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request minor Backward compatible functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants